[pull] main from github:main#49
Merged
Merged
Conversation
* chore: plan jqschema conversion to skill Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * feat: convert jqschema.sh into a skill and remove shared agentic workflow - Create .github/skills/jqschema/SKILL.md with tools, steps, and full documentation - Delete .github/workflows/shared/jqschema.md - Update all .github/workflows/*.md files to import ../skills/jqschema/SKILL.md - Update all .github/workflows/shared/*.md files to import ../skills/jqschema/SKILL.md - Update documentation references in shared workflows - Update pkg/cli/workflows/shared/jqschema.md with actual skill content - Recompile all lock files Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * feat: store jqschema.sh as a file in the skill folder Instead of generating jqschema.sh inline via a heredoc, the skill now stores jqschema.sh as a real file at .github/skills/jqschema/jqschema.sh and the setup step copies it to /tmp/gh-aw/jqschema.sh at runtime. Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * chore: recompile all lock files after jqschema skill conversion Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * chore: merge main and recompile all lock files Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…31068) * Initial plan * chore: outline console formatting fix plan Agent-Logs-Url: https://github.com/github/gh-aw/sessions/15373968-96a0-4486-9c15-f2f8fed316f9 Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com> * fix: format high-traffic cli stderr messages Agent-Logs-Url: https://github.com/github/gh-aw/sessions/15373968-96a0-4486-9c15-f2f8fed316f9 Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com> * test: harden cli stderr formatting coverage Agent-Logs-Url: https://github.com/github/gh-aw/sessions/15373968-96a0-4486-9c15-f2f8fed316f9 Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com> * style: refine cli stderr formatting assertions Agent-Logs-Url: https://github.com/github/gh-aw/sessions/15373968-96a0-4486-9c15-f2f8fed316f9 Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com> Co-authored-by: Peli de Halleux <pelikhan@users.noreply.github.com>
…acOS ARM64 / Rosetta 2 failure) (#31050) * Initial plan * chore: planning git detection fix for macOS ARM64 Agent-Logs-Url: https://github.com/github/gh-aw/sessions/82628923-30ee-4061-af73-61b141aba2e4 Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com> * fix: replace exec.Command git detection with pure Go filesystem traversal Fixes `gh aw init` failing with "not in a git repository" on macOS ARM64 (Apple Silicon) when the x86_64 binary runs under Rosetta 2. Previously, `isGitRepo()` and `FindGitRoot()` used `exec.Command("git", ...)` to detect git repositories. Under Rosetta 2 (x86_64 emulation on ARM64), this subprocess call can silently fail due to PATH differences or OS-level subprocess restrictions. Replace with pure Go filesystem traversal: walk up from the current directory (or a given starting directory) looking for a `.git` entry (directory or file, handling both normal repos and worktrees). This eliminates the dependency on the `git` executable for repository detection, making it work regardless of architecture, Rosetta translation, or git installation state. Changes: - `pkg/gitutil/gitutil.go`: `FindGitRoot()` now uses `os.Stat`/`filepath.Dir` traversal; new exported `FindGitRootFrom(startDir)` helper for callers that start from a specific directory - `pkg/cli/git.go`: `isGitRepo()` delegates to `gitutil.FindGitRoot()`; `findGitRootForPath()` uses `gitutil.FindGitRootFrom(dir)` instead of exec - `pkg/gitutil/gitutil_test.go`: add `TestFindGitRootFrom` tests - `pkg/gitutil/spec_test.go`: update doc comment to reflect new implementation Agent-Logs-Url: https://github.com/github/gh-aw/sessions/82628923-30ee-4061-af73-61b141aba2e4 Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com> * fix: address code review feedback - fix error message and improve test robustness Agent-Logs-Url: https://github.com/github/gh-aw/sessions/82628923-30ee-4061-af73-61b141aba2e4 Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com> * chore: merge main and recompile workflow lock files Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> * fix: improve FindGitRootFrom robustness and add worktree/edge-case tests Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com> Co-authored-by: Peli de Halleux <pelikhan@users.noreply.github.com> Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…ard` resolution (#31076)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )
Summary by cubic
Promotes
jqschemato a reusable skill and updates all workflows to use it. Adds ET budget control and attribution, supportsengine.modelwithoutengine.id, honorsredirect:duringaddflows, and improves git detection and CLI error output.New Features
jqschemainto a skill at.github/skills/jqschema/SKILL.mdwith a realjqschema.shscript; all workflows now import the skill.max-effective-tokens(default 10,000,000). Propagates ET usage and rate-limit signals through the agent job, detection, and failure templates; extended Claude 429/rate-limit detection and gateway log parsing.engine.modelwithoutengine.idto express model preference; validation, compiler, schema, and tests updated.gh aw addandgh aw add-wizardnow follow frontmatterredirect:for remote workflows; docs and tests added.console.Format*(e.g., compile errors, watch notices); tests added.FindGitRoot/FindGitRootFrom) to fix Rosetta/macOS ARM issues; callers updated and tests added.Migration
shared/jqschema.mdwith../skills/jqschema/SKILL.md(ensure it’s imported before components that need it).max-effective-tokens:in workflow frontmatter. Defaults apply if omitted.Written for commit 136a3fe. Summary will update on new commits.